Conversation
|
@hugovk, as an accessibility expert, what are you thoughts here? |
|
I would like to add yet another link, to languages' stats (#151). I wonder how that will play with those changes. Probably we should then split the links into separate lines. |
|
The markup could be a list instead of |
A screen reader may give a list of links, which currently would be lots of View, Contribute, View, Contribute. As a sighted user, https://translations.python.org/155/merge/ does look somewhat cluttered, as I can tell the View/Contribute in the French box relate to that translation. Here are some relevant WCAG guidelines:
We can use https://www.w3.org/WAI/WCAG22/Techniques/aria/ARIA8 The example: <h4>Neighborhood News</h4>
<p>Seminole tax hike: Seminole city managers are proposing a 75% increase in
property taxes for the coming fiscal year.
<a href="taxhike.html" aria-label="Read more about Seminole tax hike">
Read more</a>
</p>
<p>Baby Mayor: Seminole voters elect the city's youngest mayor ever by voting
in 3 year old Willy "Dusty" Williams in yesterday's mayoral election.
<a href="babymayor.html" aria-label="Read more about Seminole's new baby mayor">
Read more</a>
</p>This is similar to our situation, where we have lots of identical link texts, but they go to different pages. We can also use <a href="https://docs.python.org/fr/3/" aria-label="View French documentation" ...>View</a>
...
<a href="https://git.afpy.org/AFPy/python-docs-fr/src/branch/3.13/CONTRIBUTING.rst" aria-label="Contribute to French translations" ...>View</a> |
- move logic from template to code - use list instead of decorative characters (style tbd) - use aria labels to distinguish identical links
templates/index.html.jinja
Outdated
| @@ -13,17 +13,19 @@ | |||
| <p class="text-muted">30-day progress: {{ '{:.2f}%'.format(project.change) }}</p> | |||
|
|
|||
| <p style="text-align: center;"> | |||
There was a problem hiding this comment.
Let's drop now not needed paragraph tags. Side effect of changes is aligning this section to the left, but I'm fine with it.
Thank you for this PR!
📊 Dashboard preview 📊: https://python-docs-translations.github.io/dashboard/155/merge/